Skip to content

Sync workflow designer skill with safe-output and network reference docs#40742

Merged
pelikhan merged 2 commits into
mainfrom
copilot/add-missing-safe-output-type
Jun 22, 2026
Merged

Sync workflow designer skill with safe-output and network reference docs#40742
pelikhan merged 2 commits into
mainfrom
copilot/add-missing-safe-output-type

Conversation

Copilot AI commented Jun 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The interactive workflow designer had drifted from the canonical docs in three places: it did not map the registered dispatch-workflow safe output, it was missing 14 valid network.allowed ecosystem identifiers, and its portable HTTPS reference list omitted two in-repo guides. This updates the designer skill source and its embedded copy so generated guidance matches the current gh-aw surface area.

  • Safe output coverage

    • Added the missing mapping for workflow dispatch actions:
      • "trigger another workflow", "dispatch to workflow", "run another workflow" → dispatch-workflow

  • Network ecosystem coverage

    • Added missing mappings for:
      • chrome
      • clojure
      • dev-tools
      • julia
      • kotlin
      • latex
      • lean
      • lua
      • node-cdns
      • ocaml
      • powershell
      • python-native
      • scala
      • zig

  • Portable reference parity

    • Added the missing HTTPS references for:
      • .github/aw/subagents.md
      • .github/aw/token-optimization.md

  • Embedded prompt sync

    • Applied the same edits to both copies of the designer prompt:
      • .github/skills/agentic-workflow-designer/SKILL.md
      • pkg/cli/data/agentic_workflow_designer_skill.md

Example of the added mapping:

| "trigger another workflow", "dispatch to workflow", "run another workflow" | `dispatch-workflow` |
| "uses node CDNs (jsdelivr, unpkg)" | include `node-cdns` in `network.allowed` |

@pelikhan
Fix designer skill drift mappings
e89332c 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add missing safe output type for dispatch_workflow Sync workflow designer skill with safe-output and network reference docs Jun 22, 2026
Copilot AI requested a review from pelikhan June 22, 2026 06:45
@github-actions github-actions Bot mentioned this pull request Jun 22, 2026
Open
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Thanks for opening this Copilot agent PR to fill in the missing dispatch_workflow safe-output type and designer skill mappings — that's a well-scoped gap to address! 🛠️

A couple of things to round this out before it's ready for review:

  • Add a prose description — the body is currently a task checklist only. Once the implementation is in place, a short paragraph explaining what is missing (which safe-output type / ecosystem entries) and why it matters (what breaks or is silently skipped without it) will help reviewers orient quickly.
  • Add tests — the checklist doesn't include a step to add or update tests for the new safe-output type entry or skill mapping changes. Coverage for the new mapping (and a negative case for the previously-missing entry) would give reviewers confidence.

This is a draft with 0 changed files so far — the last three checklist items are still pending. Once those land, revisiting the above would put this in great shape.

If you'd like a hand finishing up:

Complete the remaining checklist items in PR #40742 (branch copilot/add-missing-safe-output-type):
1. Update the designer skill mappings for the missing safe output and network ecosystems in both the source and embedded skill copies.
2. Add the missing portable HTTPS references in both skill copies.
3. Add or update tests to cover the new safe-output type mapping for dispatch_workflow.
4. Add a prose description to the PR body explaining what was missing and why it matters.
5. Run `make build`, `make test-unit`, and `make agent-report-progress` to verify everything passes.

Generated by ✅ Contribution Check · 181.6 AIC · ⌖ 11 AIC · ⊞ 5.9K · ◷

@github-actions github-actions Bot mentioned this pull request Jun 22, 2026
Open
@pelikhan pelikhan marked this pull request as ready for review June 22, 2026 14:13
Copilot AI review requested due to automatic review settings June 22, 2026 14:13
@pelikhan pelikhan merged commit 0d9bca3 into main Jun 22, 2026
@pelikhan pelikhan deleted the copilot/add-missing-safe-output-type branch June 22, 2026 14:13
Copilot AI reviewed Jun 22, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates the agentic workflow designer prompt so its generated guidance matches the canonical gh-aw documentation, covering safe-output mappings, network.allowed ecosystem identifiers, and the portable HTTPS reference list.

Changes:

  • Adds a safe-output mapping for dispatching/triggering workflows (dispatch-workflow).
  • Expands network.allowed guidance with 14 additional ecosystem identifiers.
  • Adds two missing portable HTTPS references and syncs changes across both prompt copies.
Show a summary per file
File Description
pkg/cli/data/agentic_workflow_designer_skill.md Updates the embedded designer prompt (safe-output mapping, network.allowed ecosystems, and portable references).
.github/skills/agentic-workflow-designer/SKILL.md Updates the skill source prompt to match canonical docs (same prompt edits as the embedded copy).

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 3

Comment thread pkg/cli/data/agentic_workflow_designer_skill.md
| "request reviewer", "hide comment" | `add-reviewer`, `hide-comment` |
| "create/update project", "project status update" | `create-project`, `update-project`, `create-project-status-update` |
| "update release", "upload release asset" | `update-release`, `upload-asset` |
| "trigger another workflow", "dispatch to workflow", "run another workflow" | `dispatch-workflow` |
Comment thread pkg/cli/data/agentic_workflow_designer_skill.md
| "uses Julia packages" | include `julia` in `network.allowed` |
| "uses Kotlin / JetBrains packages" | include `kotlin` in `network.allowed` |
| "uses LuaRocks / Lua packages" | include `lua` in `network.allowed` |
| "uses node CDNs (jsdelivr, unpkg)" | include `node-cdns` in `network.allowed` |
Comment thread .github/skills/agentic-workflow-designer/SKILL.md
| "uses Julia packages" | include `julia` in `network.allowed` |
| "uses Kotlin / JetBrains packages" | include `kotlin` in `network.allowed` |
| "uses LuaRocks / Lua packages" | include `lua` in `network.allowed` |
| "uses node CDNs (jsdelivr, unpkg)" | include `node-cdns` in `network.allowed` |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@pelikhan pelikhan pelikhan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@pelikhan pelikhan

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pelikhan